Skip to content

Conversation

jlalmes
Copy link

@jlalmes jlalmes commented Jul 3, 2025

☕️ Reasoning

Fix for bug introduced by #11470

Context here: #10244 (comment)

  • Added new internal _getSession function that broadcasts on the same BroadcastChannel the message listener is on, therefore not re-triggering a fresh fetch to GET /session.
  • Updated references to use this new function.
  • Kept external-facing getSession behaviour so that the user can force update the session.

🧢 Checklist

  • Documentation
  • Tests
  • Ready to be merged

🎫 Affected issues

Fixes: #10244

📌 Resources

@jlalmes jlalmes requested a review from ThangHuuVu as a code owner July 3, 2025 01:15
Copy link

vercel bot commented Jul 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
auth-docs Ready Ready Preview Comment Aug 30, 2025 9:21am
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
next-auth-docs Ignored Ignored Preview Aug 30, 2025 9:21am

Copy link

vercel bot commented Jul 3, 2025

@jlalmes is attempting to deploy a commit to the authjs Team on Vercel.

A member of the Team first needs to authorize it.

@jlalmes
Copy link
Author

jlalmes commented Jul 3, 2025

@balazsorban44 Can you please review & release! 🙏

@albertehc
Copy link

some news on this PR pls? getting 2x calls on production environments

@jlalmes
Copy link
Author

jlalmes commented Aug 30, 2025

@ThangHuuVu @balazsorban44, can you take a look please 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SessionProvider sends many requests to retrieve the session
2 participants